Skip to content

Conversation

@saifsultanc
Copy link
Contributor

Context

⛑️ Ticket(s): https://secure.helpscout.net/conversation/2740666744/72953

Summary

The Disable Submit Button Until Required Fields Are Filled Out snippet is conflicting with GPPA, causing the form to break. Populated fields just pulse and don't update.

Matt A's loom explaining the issue:
https://www.loom.com/share/e98e2369db5a43278f5d2706612fad54

After the update (fix):
https://www.loom.com/share/76f4bf19d9554486bde1a39e74e97ea4

…Fixed a compatibility issue with GP Populate Anything.
@saifsultanc saifsultanc added the bug Something isn't working label Oct 25, 2024
Comment on lines +35 to +36
add_action( 'wp_footer', array( $this, 'output_script' ) );
add_action( 'gform_preview_footer', array( $this, 'output_script' ) );
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was this to prevent AJAX errors?

Could we use gform_print_scripts instead?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, and I will check gform_print_scripts

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Doesnt work with gform_print_scripts, i think its because it not firing at the correct time.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works for me considering this is a snippet. Thanks for checking!

Comment on lines +35 to +36
add_action( 'wp_footer', array( $this, 'output_script' ) );
add_action( 'gform_preview_footer', array( $this, 'output_script' ) );
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works for me considering this is a snippet. Thanks for checking!

@saifsultanc saifsultanc merged commit 0394642 into master Oct 28, 2024
4 checks passed
@saifsultanc saifsultanc deleted the saif/fix/72953-fix-gppa-disable-submit branch October 28, 2024 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Development

Successfully merging this pull request may close these issues.

3 participants